Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show hidden map if it's requested by URL wtracks?map=... #27

Merged
merged 2 commits into from
Dec 24, 2021

Conversation

eddy-geek
Copy link
Contributor

@eddy-geek eddy-geek commented Dec 22, 2021

Commit 1

Show hidden map if it's requested by URL wtracks?map=...

It is only shown transiently:

  • since right now ?map param is stored in wt.baseLayer, it will still be shown on next refresh (could be changed I guess)
  • but as soon as user changes it, it will be hidden again (since wt.mymaps is not changed)

Commit 2

Add ?overlays request parameter to control overlays

Behaviour on refresh and for hidden overlays is similar as for ?map...
... except I think there is a preexisting issue right now where enabled overlays are not shown on refresh, but they are in the shared URL.

Examples:

Cc #26

It is only shown transiently:
* since right now map is remembered, it will still be shown on next refresh (could be changed)
* but as soon as user changes it, it will be hidden again
Behaviour on refresh and for hidden overlays is similar as for `?map`...
... except there is a preexisting issue right now where enabled overlays are not shown on refresh, but they are in the shared URL.

Examples:
- Multi: http://localhost:8000/?map=OpenStreetMap&overlays=Hills,Cycling
- Hidden: http://localhost:8000/?map=OpenStreetMap&overlays=FR%20IGN%20Parcelles

Part of opoto#26
@opoto
Copy link
Owner

opoto commented Dec 24, 2021

Good, thanks!

@opoto opoto merged commit 3be19ba into opoto:master Dec 24, 2021
eddy-geek added a commit to eddy-geek/wtracks that referenced this pull request Dec 25, 2021
opoto added a commit that referenced this pull request Dec 25, 2021
Fix overlays restore (regression from #27)
@eddy-geek eddy-geek deleted the nits branch December 27, 2021 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants